Skip to content

feat: update mparticle versions#33

Merged
jamesnrokt merged 13 commits into
mainfrom
feat/SQDSDKS-7524-update-mparticle-versions
Jul 24, 2025
Merged

feat: update mparticle versions#33
jamesnrokt merged 13 commits into
mainfrom
feat/SQDSDKS-7524-update-mparticle-versions

Conversation

@jamesnrokt

@jamesnrokt jamesnrokt commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

Instructions

  1. PR target branch should be against development
  2. PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
  3. PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

  • Updated internal versions of mParticle SDK
  • Made example app buildable while retaining the AppDelegate and MainActivity examples
  • Fixed issue with MPGDPRConsent and MPCCPAConsent caused by Obj-c to Swift conversion in 8.27.2
  • Fix Invalid (non-string) key crash in getUserIdentities
  • Added build commands to Github actions workflow
  • Added a clean script to example app for easier reruns
  • Added README for example applications

Testing Plan

  • Ran both example applications and verified data processed into UI
Screenshot 2025-07-23 at 3 43 11 pm
  • Added build commands to Github actions workflow

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

Copilot AI review requested due to automatic review settings July 23, 2025 05:39
@jamesnrokt
jamesnrokt requested a review from a team as a code owner July 23, 2025 05:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the MParticle SDK versions for both Android and iOS platforms and improves the example application's buildability. The key changes include fixing compatibility issues with the newer iOS SDK version, particularly around GDPR/CCPA consent handling and user identity enumeration, while also enhancing the development workflow with build automation and documentation.

  • Updated Android SDK from 5.38.1 to 5.71.0 and iOS SDK to 8.35.0
  • Fixed iOS compatibility issues with identity type conversion and consent handling
  • Enhanced example app with build scripts, documentation, and CI/CD improvements

Reviewed Changes

Copilot reviewed 11 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
plugin/src/ios/CDVMParticle.m Fixed identity enumeration crash by converting enum keys to strings and updated import to use module syntax
plugin/plugin.xml Updated Android/iOS SDK versions and modernized iOS podspec configuration
example/www/js/index.js Enhanced example with better logging and Cordova-specific naming
example/scripts/clean.js Added utility script for cleaning and resetting project state
example/platform_overrides/ios/AppDelegate.m Standardized API key placeholders for consistency
example/platform_overrides/android/MainActivity.java Cleaned up imports and standardized API key placeholders
example/package.json Updated Cordova platform versions and added clean script
example/hooks/after_platform_add/010_copy_platform_specific_files.js Added hook for copying platform-specific override files
example/config.xml Added iOS deployment target and platform copy hook
example/README.md Added comprehensive documentation for setup and usage
.github/workflows/pull-request.yml Updated GitHub Actions versions and added example app build jobs
Files not reviewed (2)
  • example/package-lock.json: Language not supported
  • plugin/package-lock.json: Language not supported

Comment thread plugin/src/ios/CDVMParticle.m Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jamesnrokt
jamesnrokt merged commit 0ef4a93 into main Jul 24, 2025
11 checks passed
@jamesnrokt
jamesnrokt deleted the feat/SQDSDKS-7524-update-mparticle-versions branch July 24, 2025 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants